Skip to content

CLDR-19275 move webdriver to be a proper subproject#5463

Draft
srl295 wants to merge 3 commits intomainfrom
srl295/cldr-19275/spotless-java-version
Draft

CLDR-19275 move webdriver to be a proper subproject#5463
srl295 wants to merge 3 commits intomainfrom
srl295/cldr-19275/spotless-java-version

Conversation

@srl295
Copy link
Member

@srl295 srl295 commented Mar 14, 2026

  • change webdriver main to be a main() instead of a test
  • allow webdriver to inherit versions from parent

The PR #5403 pointed out that we have two copies of the spotless machinery. This starts to reduce that.

CLDR-19275

  • This PR completes the ticket.

ALLOW_MANY_COMMITS=true

- change webdriver main to be a main() instead of a test
- allow webdriver to inherit versions from parent
@srl295 srl295 requested a review from btangmu March 14, 2026 00:43
@srl295 srl295 self-assigned this Mar 14, 2026
@cloudflare-workers-and-pages
Copy link

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
cldr-next 946e409 Commit Preview URL

Branch Preview URL
Mar 14 2026, 12:43 AM

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 14, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
cldr-next 9675f7d Commit Preview URL

Branch Preview URL
Mar 14 2026, 12:53 AM

@srl295
Copy link
Member Author

srl295 commented Mar 14, 2026

builds, doesn't work yet.

@btangmu
Copy link
Member

btangmu commented Mar 20, 2026

builds, doesn't work yet.

How does it fail? Should I investigate?

@srl295
Copy link
Member Author

srl295 commented Mar 20, 2026

@btangmu you can see the failure https://github.com/unicode-org/cldr/actions/runs/23076569629/job/67038129961#step:11:26 - i think I just didn't wire up the test properly yet.

THe point of this is to normalize the webdriver's build so it's part of the other projects, rather than separate (as if it were still in a separate repo) so that it can benefit from dependency updates etc. It had its own copy of the spotless config for example.

@btangmu
Copy link
Member

btangmu commented Mar 20, 2026

There are error messages from setup-config.sh involving /tmp/cldr-code.jar -- not clear whether these are central to the main failure.

setup-config.sh runs in cldr-apps/Dockerfile, not cldr-apps-webdriver/Dockerfile. So it's cldr-apps, not cldr-apps-webdriver, that fails here. That suggests the error is "invalid address of the remote server" rather than "browser start-up failure".

This command has two parts:

docker compose run --rm webdriver || ( docker compose exec -it cldr-apps tail /logs/messages.log ; true )

Seems like it's the second part (docker compose exec -it cldr-apps...) that's failing if the setup-config.sh errors are the cause. That would imply the first part (docker compose run --rm webdriver) succeeds, in which case I guess the browser starts up OK...

# See tools/cldr-apps/README.md
run: >
docker compose run -v ~/.m2/repository:/root/.m2/repository:rw --rm webdriver || ( docker compose exec -it cldr-apps tail /logs/messages.log ; true )
docker compose run --rm webdriver || ( docker compose exec -it cldr-apps tail /logs/messages.log ; true )
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

main failure occurs here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

org.openqa.selenium.SessionNotCreatedException "Could not start a new session. Possible causes are invalid address of the remote server or browser start-up failure." caused by java.nio.channels.ClosedChannelException "I/O operation on a channel that has already been closed"

@btangmu
Copy link
Member

btangmu commented Mar 20, 2026

Why is it that webdriver runs before cldr-apps? Could cldr-apps start first instead?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants